Skip to content

ProblemSets: Place set description in info button.#2925

Merged
pstaabp merged 1 commit intoopenwebwork:developfrom
somiaj:description-info-icon
Mar 10, 2026
Merged

ProblemSets: Place set description in info button.#2925
pstaabp merged 1 commit intoopenwebwork:developfrom
somiaj:description-info-icon

Conversation

@somiaj
Copy link
Contributor

@somiaj somiaj commented Mar 3, 2026

Move the set description from a tooltip on the link to the set to a tooltip on an info button located just right of the set link if the set description is not empty.

This adds an additional option to the set-id-tooltip class JavaScript that if data-fallback-placements is set as a spaced separated list, that list will be passed to the bootstrap Tooltip on creation.

This was suggested by @dlglin.

@somiaj somiaj force-pushed the description-info-icon branch 3 times, most recently from dd9a991 to 8b3149c Compare March 10, 2026 18:16
Copy link
Member

@pstaabp pstaabp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Copy link
Member

@dlglin dlglin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me.

Move the set description from a tooltip on the link to the set to
a tooltip on an info button located just right of the set link if
the set description is not empty.

This adds an additional option to the `set-id-tooltip` class
JavaScript that if `data-fallback-placements` is set as a spaced
separated list list, that list will be passed to the bootstrap
Tooltip on creation.
@somiaj somiaj force-pushed the description-info-icon branch from 8b3149c to c32fda0 Compare March 10, 2026 19:18
@pstaabp pstaabp merged commit de5edce into openwebwork:develop Mar 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants